Skip to content

chore: remove agent version pinning in the CI #9106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Jul 28, 2025
Merged

chore: remove agent version pinning in the CI #9106

merged 2 commits into from
Jul 28, 2025

Conversation

@pkoutsovasilis
Copy link
Contributor

@pkoutsovasilis pkoutsovasilis commented Jul 23, 2025 •
edited
Loading

What does this PR do?

This PR removes the pinned AGENT_VERSION, AGENT_PACKAGE_VERSION, and BEAT_VERSION values from the CI environment and updates the .package-version file to 9.0.5.

It also re-enables the upgrade_broken_package_test.go test that was previously skipped due to reliance on pinned versions, since that constraint is now resolved.

PS: as discussed here a review from observablt-ci and observablt-ci-contractors is not required (cc @ebeahan @pierrehilbert )

Why is it important?

The version pinning was introduced as a temporary workaround to support the 9.0.5 release. Now that 9.0.5 DRA is released, we are reverting to an unpinned state.

Checklist

  • I have read and understood the pull request guidelines of this project.
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

None. This only impacts internal CI setup and test gating. End users are not affected.

How to test this PR locally

Open a draft PR 🙂

Related issues

  • N/A

@pkoutsovasilis pkoutsovasilis self-assigned this Jul 23, 2025
@pkoutsovasilis pkoutsovasilis mentioned this pull request Jul 23, 2025
Merged
@mergify
Copy link
Contributor

mergify bot commented Jul 26, 2025

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b ci/90_revert_agent_pinning upstream/ci/90_revert_agent_pinning
git merge upstream/9.0
git push upstream ci/90_revert_agent_pinning

@pkoutsovasilis pkoutsovasilis force-pushed the ci/90_revert_agent_pinning branch from 35d8b08 to 77fa24e Compare July 28, 2025 03:08
@elasticmachine
Copy link
Collaborator

elasticmachine commented Jul 28, 2025 •
edited
Loading

💛 Build succeeded, but was flaky

Failed CI Steps

History

cc @pkoutsovasilis

@elastic-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@pkoutsovasilis pkoutsovasilis marked this pull request as ready for review July 28, 2025 09:10
@pkoutsovasilis pkoutsovasilis requested review from a team as code owners July 28, 2025 09:10
@pkoutsovasilis pkoutsovasilis requested review from pchila and straistaru and removed request for a team July 28, 2025 09:10
@elasticmachine
Copy link
Collaborator

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

pchila
pchila approved these changes Jul 28, 2025
View reviewed changes
Copy link
Member

@pchila pchila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pkoutsovasilis pkoutsovasilis merged commit da48a75 into elastic:9.0 Jul 28, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@swiatekm swiatekm swiatekm approved these changes

@pchila pchila pchila approved these changes

@straistaru straistaru Awaiting requested review from straistaru straistaru is a code owner automatically assigned from elastic/elastic-agent-control-plane

Assignees

@pkoutsovasilis pkoutsovasilis

Labels

backport-skip skip-changelog Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pkoutsovasilis @elasticmachine @swiatekm @pchila